Skip to content

Conversation

@sfc-gh-yixie
Copy link
Collaborator

Please answer these questions before submitting your pull requests. Thanks!

  1. What GitHub issue is this PR addressing? Make sure that there is an accompanying issue to your PR.

    Fixes #NNNN

  2. Fill out the following pre-review checklist:

    • I am adding a new automated test(s) to verify correctness of my new code
    • I am adding new logging messages
    • I am adding a new telemetry message
    • I am modifying authorization mechanisms
    • I am adding new credentials
    • I am modifying OCSP code
    • I am adding a new dependency
  3. Please describe how your code solves the related issue.

    Please write a short description of how your code change solves the related issue.

  4. (Optional) PR for stored-proc connector:

Copy link

@keller00 keller00 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Don't forget to bump the version and re-dump requirements files

Copy link
Collaborator

@sfc-gh-mkeller sfc-gh-mkeller left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Assuming @keller00 knows what he's doing

🚢 🚢

@sfc-gh-yixie sfc-gh-yixie merged commit 5575562 into main Jan 30, 2025
94 checks passed
@sfc-gh-yixie sfc-gh-yixie deleted the yixie-fix-scoped-temp-stage branch January 30, 2025 04:49
@github-actions github-actions bot locked and limited conversation to collaborators Jan 30, 2025
@sfc-gh-yixie sfc-gh-yixie changed the title Changed not to use scoped temp stage SNOW-1903407: Changed not to use scoped temp stage Jan 30, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants